Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

samples: enable timeout for sidewalk msg put #638

Merged
merged 1 commit into from
Oct 30, 2024
Merged

samples: enable timeout for sidewalk msg put #638

merged 1 commit into from
Oct 30, 2024

Conversation

ktaborowski
Copy link
Contributor

@ktaborowski ktaborowski commented Oct 29, 2024 •
edited
Loading

KRKNWK-19515

CI parameters

Github_actions:
  #(branch, hash, pull/XXX/head)
  NRF_revision: v2.8-branch
  # Do not change after creating PR
  Create_NRF_PR: false
Jenkins:
  test-sdk-sidewalk: master

Description

JIRA ticket:

Self review

  • There is no commented code.
  • There are no TODO/FIXME comments without associated issue ticket.
  • Commits are properly organized.
  • Change has been tested.
  • Tests were updated (if applicable).

@github-actions github-actions bot added the source PR changing src files label Oct 29, 2024
@ktaborowski ktaborowski added func-nightly and removed source PR changing src files labels Oct 29, 2024
@github-actions github-actions bot added the source PR changing src files label Oct 29, 2024
@ktaborowski
samples: enable timeout for sidewalk msg put
2f99eb9 
[KRKNWK-19515]

Signed-off-by: Krzysztof Taborowski <[email protected]>
@github-actions GitHub Actions
Copy link

Sample diff used total
nrf54l15dk/nrf54l15/cpuapp:sample.sidewalk.dut RAM 0 B 140.79 KB 0 B
ROM 12 B 510.07 KB 0 B
nrf54l15dk/nrf54l15/cpuapp:sample.sidewalk.dut.no_secure RAM 0 B 140.77 KB 0 B
ROM 12 B 499.21 KB 0 B
nrf54l15dk/nrf54l15/cpuapp:sample.sidewalk.hello.ble_only RAM 0 B 92.3 KB 0 B
ROM 12 B 365.2 KB 0 B
nrf54l15dk/nrf54l15/cpuapp:sample.sidewalk.hello.ble_only.release RAM 0 B 87.67 KB 0 B
ROM 12 B 296.18 KB 0 B
nrf54l15dk/nrf54l15/cpuapp:sample.sidewalk.hello.release RAM 0 B 102.46 KB 0 B
ROM 12 B 372.02 KB 0 B
thingy53/nrf5340/cpuapp:sample.sidewalk.demo.ble_only RAM 0 B 106.31 KB 0 B
ROM 12 B 351.21 KB 0 B
nrf52840dk/nrf52840:sample.sidewalk.demo RAM 0 B 118 KB 0 B
ROM 12 B 450.54 KB 0 B
nrf52840dk/nrf52840:sample.sidewalk.demo.ble_only RAM 0 B 102.93 KB 0 B
ROM 12 B 368.17 KB 0 B
nrf52840dk/nrf52840:sample.sidewalk.dut RAM 0 B 141.65 KB 0 B
ROM 12 B 515.59 KB 0 B
nrf52840dk/nrf52840:sample.sidewalk.dut.no_secure RAM 0 B 141.62 KB 0 B
ROM 12 B 507.05 KB 0 B
nrf52840dk/nrf52840:sample.sidewalk.hello.ble_only RAM 0 B 92.76 KB 0 B
ROM 12 B 366.51 KB 0 B
nrf52840dk/nrf52840:sample.sidewalk.hello.ble_only.release RAM 0 B 87.95 KB 0 B
ROM 12 B 299.31 KB 0 B
nrf52840dk/nrf52840:sample.sidewalk.hello.release RAM 0 B 103.03 KB 0 B
ROM 12 B 378.25 KB 0 B
nrf5340dk/nrf5340/cpuapp:sample.sidewalk.demo RAM 0 B 111.48 KB 0 B
ROM 12 B 395.24 KB 0 B
nrf5340dk/nrf5340/cpuapp:sample.sidewalk.demo.ble_only RAM 0 B 96.56 KB 0 B
ROM 12 B 311.74 KB 0 B
nrf5340dk/nrf5340/cpuapp:sample.sidewalk.dut RAM 0 B 135.25 KB 0 B
ROM 12 B 461.03 KB 0 B
nrf5340dk/nrf5340/cpuapp:sample.sidewalk.dut.no_secure RAM 0 B 135.23 KB 0 B
ROM 12 B 453.58 KB 0 B
nrf5340dk/nrf5340/cpuapp:sample.sidewalk.hello.ble_only RAM 0 B 86.64 KB 0 B
ROM 12 B 310.32 KB 0 B
nrf5340dk/nrf5340/cpuapp:sample.sidewalk.hello.ble_only.release RAM 0 B 82 KB 0 B
ROM 12 B 240.87 KB 0 B
nrf5340dk/nrf5340/cpuapp:sample.sidewalk.hello.release RAM 0 B 96.92 KB 0 B
ROM 12 B 320.6 KB 0 B
nrf54l15dk/nrf54l15/cpuapp:sample.sidewalk.demo RAM 0 B 117.04 KB 0 B
ROM 12 B 444.51 KB 0 B
nrf54l15dk/nrf54l15/cpuapp:sample.sidewalk.demo.ble_only RAM 0 B 102.26 KB 0 B
ROM 12 B 366.88 KB 0 B

totyz
totyz approved these changes Oct 30, 2024
View reviewed changes
Copy link
Contributor

@totyz totyz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nightly test looks very good

@ktaborowski ktaborowski merged commit ae200c2 into nrfconnect:main Oct 30, 2024
59 checks passed
@ktaborowski ktaborowski deleted the symbols branch November 8, 2024 11:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@totyz totyz totyz approved these changes

@RobertGalatNordic RobertGalatNordic Awaiting requested review from RobertGalatNordic

Assignees
No one assigned
Labels
func-integration source PR changing src files
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ktaborowski @totyz